Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Zipkin - develop #9631

Merged
merged 10 commits into from
Nov 9, 2020
Merged

Zipkin - develop #9631

merged 10 commits into from
Nov 9, 2020

Conversation

heifner
Copy link
Contributor

@heifner heifner commented Nov 3, 2020

Change Description

  • Add new zipkin support for sending trace span messages to zipkin.
  • Enabled by new chain_plugin config options for nodeos and cloner_plugin config options for rodeos
    • telemetry-url
      "Send Zipkin spans to url. e.g. http://127.0.0.1:9411/api/v2/spans"
    • telemetry-service-name
      "Zipkin localEndpoint.serviceName sent with each span"
    • telemetry-timeout-us Default: 200000
      "Timeout for sending Zipkin span."

Zipkin sends zipkin Spans to zipkin via http json on a separate thread. timeout_us can be configured for how long to allow each http post before hitting the time out. Nine consecutive failures will disable the zipkin sender until a restart of application.

See #9358 for example screen shots.

Change Type

Select ONE

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

Example of querying a transaction by id:
http://localhost:9411/api/v2/traces?annotationQuery=trx_id=a6c5878d84c2e52f750f50f4b314ff920629e6bf01b753e86896fd1e03577d35

@nksanthosh nksanthosh requested a review from jgiszczak November 5, 2020 19:48
@heifner
Copy link
Contributor Author

heifner commented Nov 6, 2020

Currently working on moving this support outside the logging framework.

@heifner heifner marked this pull request as draft November 6, 2020 12:45
@heifner heifner changed the title Zipkin appender - develop Zipkin - develop Nov 6, 2020
@heifner heifner marked this pull request as ready for review November 6, 2020 15:04
@heifner heifner merged commit 94c98f1 into develop Nov 9, 2020
@heifner heifner deleted the zipkin-appender-dev branch November 9, 2020 21:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants